Skip to content

Add signed add-on capability and adjust max rating#11703

Merged
bramkragten merged 4 commits intodevfrom
Add-signed-add-on-capability-and-adjust-max-rating
Feb 17, 2022
Merged

Add signed add-on capability and adjust max rating#11703
bramkragten merged 4 commits intodevfrom
Add-signed-add-on-capability-and-adjust-max-rating

Conversation

@ludeeus
Copy link
Copy Markdown
Member

@ludeeus ludeeus commented Feb 16, 2022

Breaking change

Proposed change

home-assistant/supervisor#3450 (comment)

Screenshot from 2022-02-16 18-00-49
Screenshot from 2022-02-16 18-00-56

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Example configuration

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue or discussion:
  • Link to documentation pull request:

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

@ludeeus ludeeus added the Supervisor Related to the supervisor panel label Feb 16, 2022
Comment thread hassio/src/addon-view/info/hassio-addon-info.ts Outdated
bramkragten
bramkragten previously approved these changes Feb 16, 2022
Comment thread hassio/src/addon-view/info/hassio-addon-info.ts Outdated
green: [5, 6].includes(Number(this.addon.rating)),
yellow: [3, 4].includes(Number(this.addon.rating)),
green: [6, 7, 8].includes(Number(this.addon.rating)),
yellow: [3, 4, 5].includes(Number(this.addon.rating)),
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This yellow could be a bit darker. It's hard to read with white text.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's using this

/* for label-badge */
--label-badge-red: #DF4C1E;
--label-badge-blue: #039be5;
--label-badge-green: #0DA035;
--label-badge-yellow: #f4b400;
--label-badge-grey: #9e9e9e;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

alright let's keep it for now.

Comment thread hassio/src/addon-view/info/hassio-addon-info.ts Outdated
balloob
balloob previously approved these changes Feb 17, 2022
@bramkragten bramkragten merged commit cbd0ef6 into dev Feb 17, 2022
@bramkragten bramkragten deleted the Add-signed-add-on-capability-and-adjust-max-rating branch February 17, 2022 09:43
@github-actions github-actions bot locked and limited conversation to collaborators Feb 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed Supervisor Related to the supervisor panel

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants